Skip to content

Conversation

@fw-bot
Copy link
Collaborator

@fw-bot fw-bot commented Dec 16, 2021

  1. active is just a deprecated field that means exactly the same as
    auto_install.
  2. auto_install can be a list.

See https://github.com/odoo/odoo/blob/6cb3140ea8f62a265b76ae494295f466ca6a00c4/odoo/modules/module.py#L348-L352

Original PR: #1109

Co-authored-by: Antoine Vandevenne (anv) anv@odoo.com

Forward-Port-Of: #1427

@robodoo
Copy link
Collaborator

robodoo commented Dec 16, 2021

@fw-bot
Copy link
Collaborator Author

fw-bot commented Dec 16, 2021

Ping @AntoineVDV cherrypicking of pull request #1427 failed.

stderr:

11:32:30.368657 git.c:344               trace: built-in: git cherry-pick 5b90162e59082bf346f3a998567d5b9e7ae7c40d
error: could not apply 5b90162e... [IMP] developer/modules: deprecate 'active' and detail 'auto_install'
hint: after resolving the conflicts, mark the corrected paths
hint: with 'git add <paths>' or 'git rm <paths>'
hint: and commit the result with 'git commit'
----------
status:

Either perform the forward-port manually (and push to this branch, proceeding as usual) or close this PR (maybe?).

In the former case, you may want to edit this PR message as well.

1. `active` is just a deprecated field that means exactly the same as
   `auto_install`.
2. `auto_install` can be a list.

See https://github.com/odoo/odoo/blob/6cb3140ea8f62a265b76ae494295f466ca6a00c4/odoo/modules/module.py#L348-L352

Original PR: #1109

X-original-commit: 03a54e5
Co-authored-by: Antoine Vandevenne (anv) <anv@odoo.com>
@AntoineVDV AntoineVDV force-pushed the 15.0-13.0-auto_install_list-anv-8lak-fw branch from c0e049a to 21f8b06 Compare December 16, 2021 10:43
@AntoineVDV
Copy link
Collaborator

@robodoo r+

@C3POdoo C3POdoo requested a review from a team December 16, 2021 10:45
@robodoo robodoo closed this in 53cb375 Dec 16, 2021
@robodoo robodoo temporarily deployed to merge December 16, 2021 10:51 Inactive
@fw-bot fw-bot deleted the 15.0-13.0-auto_install_list-anv-8lak-fw branch December 30, 2021 11:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants